fix #283 - feat: Reduce warnings in build and test logs - #323

• pnpm-lock.yaml: Generated file

• pnpm-lock.yaml: Generated file

packages/open-workflow-diagram-editor/tests/react-flow/edges/Edges.test.tsx:216

• Using svg!.firstChild is brittle because firstChild can be a whitespace/text node depending on how JSX is compiled/rendered. Prefer snapshotting the first element child to avoid flakey/incorrect snapshots.

    const svg = container.querySelector("svg");
    expect(svg).not.toBeNull();
    expect(svg!.firstChild).toMatchSnapshot();

packages/open-workflow-diagram-editor/stories/features/DiagramEditorErrorBoundary.stories.tsx:42

• Capturing console.error into a variable can lose its this binding in some runtimes. Binding it to console avoids edge-case failures when forwarding to the original implementation from inside the spy.

    const originalConsoleError = console.error;

• pnpm-lock.yaml: Generated file

packages/open-workflow-diagram-editor/stories/features/DiagramEditorErrorBoundary.stories.tsx:52

• The mockImplementation contains an overly long condition with missing spacing around ===, and it calls the extracted console.error function without an explicit receiver. Formatting this improves readability and helps avoid potential lint failures; using .apply(console, ...) preserves the console method binding in all runtimes.

    const consoleErrorSpy = spyOn(console, "error").mockImplementation((...args) => {
      const error = args[1];

      if (error instanceof Error && (error.message===DEFAULT_ERROR_MESSAGE || error.message===CUSTOM_ERROR_MESSAGE)) {
        return;

• pnpm-lock.yaml: Generated file

packages/open-workflow-diagram-editor/stories/helpers.ts:20

• DiagramEditor is imported using import type, but it’s then referenced in a typeof type query (StoryObj<typeof DiagramEditor>). In TypeScript, typeof in a type position still requires the symbol to exist in the value namespace, so this pattern typically fails with “Cannot use 'DiagramEditor' as a value because it was imported using 'import type'”. Use a module type query instead (or import DiagramEditor as a value).

import type { DiagramEditor } from "./features/DiagramEditor";

type Story = StoryObj<typeof DiagramEditor>;

packages/open-workflow-diagram-editor/tests/core/taskDetails.test.ts:274

• Grammar nit in the newly added eslint-disable reason: “a Open” should be “an Open”.

        // eslint-disable-next-line unicorn/no-thenable -- then is a Open Workflow Spec field

packages/open-workflow-diagram-editor/tests/side-panel/NodeDetailsView.test.tsx:39

• Grammar nit in the newly added eslint-disable reason: “a Open” should be “an Open”.

        // eslint-disable-next-line unicorn/no-thenable -- then is a Open Workflow Spec field

• pnpm-lock.yaml: Generated file

packages/open-workflow-diagram-editor/src/components/ui/combobox.tsx:276

• ComboboxChipsInput no longer strips children out of the props spread. If a caller passes children (even accidentally via composition), it will be forwarded into ComboboxPrimitive.Input which likely renders an <input>; React will warn on children for void elements (and it can reintroduce the warnings this PR is trying to eliminate).

function ComboboxChipsInput({ className, ...props }: ComboboxPrimitive.Input.Props) {
  return (
    <ComboboxPrimitive.Input
      data-slot="combobox-chip-input"
      className={cn("dec:min-w-16 dec:flex-1 dec:outline-none", className)}
      {...props}
    />